Skip to content

Conversation

@dot-agi
Copy link
Member

@dot-agi dot-agi commented Mar 14, 2025

Closes #838

@dot-agi dot-agi requested a review from areibman March 14, 2025 22:58
@dot-agi dot-agi self-assigned this Mar 14, 2025
@teocns
Copy link
Contributor

teocns commented Mar 14, 2025

does it not work on dev branch?

Nevertheless this does not go in the right direction

@teocns teocns closed this Mar 14, 2025
@dot-agi
Copy link
Member Author

dot-agi commented Mar 14, 2025

Does not work on dev.

@devin-ai-integration
Copy link
Contributor

I've updated the PR to fix the issue where session URL wasn't being displayed when using agentops.init() only.

The root cause was that when auto-starting a session, the tags from the config weren't being passed to start_session(). This meant that:

  1. The session was created without tags
  2. The session URL was displayed but without proper context

Changes made:

  • Modified client.py to pass tags from config when auto-starting sessions
  • Kept the recursion prevention logic to avoid infinite loops
  • Maintained the default log level change from WARNING to INFO to ensure visibility of session URLs

This change ensures that when a user calls agentops.init() with auto_start_session=True (now the default), they'll see the session URL in the logs with all the proper tags applied.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

URL doesn't print in terminal when running with only agentops.init

3 participants